-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Add recipes for dmri-amico and dmri-dicelib #24708
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/dmri-amico:
For recipes/dmri-dicelib:
|
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
|
Happy to be a maintainer! Thanks @pauldmccarthy |
|
Me, too, thanks! |
7f8722b to
f6f7619
Compare
|
@conda-forge/help-python, ready for review! Posting this message now, as I'm going to be offline for the next couple of days. The Windows builds were previously succeeding, but the latest one has failed due to what I believe is a transient error, coming from git, during the Configuring conda stage, and unrelated to these recipes (note the invalid file path - this is running on Windows): I was also experiencing a strange issue earlier (azure log here, relevant section also copied below in Details |
f6f7619 to
82fe50d
Compare
…ng to work around this by pinning numpy at build/run time
82fe50d to
f773e50
Compare
|
Hi @conda-forge/help-python, just a gentle reminder that this recipe is ready for [re-]review - thanks! |
|
Hi @conda-forge/help-python, another poke to try and get this recipe moving - thanks! |
Howdy, this PR adds two recipes - one for
dmri-amico, along with a core library that it depends on -dmri-dicelib:AMICO is a popular diffusion MRI microstructure modelling framework.
@daducci and @nightwnvol - would you be able to leave a comment here, confirming that you are happy to be added as package maintainers? Thanks!
Checklist
url) rather than a repo (e.g.git_url) is used in your recipe (see here for more details).